feat: DAG-aligned improvements (v2.6.0)#17
Merged
Conversation
Three features: Critical Path Identification, Task-Level Resume/Caching, and Early Integration Checkpoints — all sharing a centralized task-graph.json workspace file as their DAG engine.
15 tasks across 3 chunks: scripts+tests, documentation, release. Covers compute-critical-path.sh, detect-resume.sh, check-integration-point.sh, task-graph.json schema, and updates to all 19 modified files.
String comparison of ISO 8601 timestamps fails across timezones (git log returns local TZ, completed_at may be UTC). Convert both to epoch seconds via date command for correct comparison.
- detect-resume.sh: add "missing" file classification for deleted outputs - detect-resume.sh: use printf %b to prevent format-specifier injection - README.md: update hook count from "Five" to "Eight"
compute-critical-path.sh: - CRITICAL: add cycle detection pass before DFS (prevents infinite recursion) - HIGH: add null guard for non-existent dependency references - HIGH: skip NEXT_CRITICAL in blocked-task loop (prevents duplicate warnings) - LOW: fix "all parallel" message when all tasks complete shared-phases.md: - Add 3 new hooks to Hooks inventory section
- README.md: fix hook count "Eight" → "Nine" - CLAUDE.md: fix test count "9 files (78 assertions)" → "12 files (145 assertions)" - CLAUDE.md: fix "Adding a New Hook" to reference shared-phases.md (not SKILL.md) - coordination-patterns.md: sync Lead Processing table with shared-phases.md (add task-graph.json update to COMPLETED, add PROGRESS/CHECKPOINT/integration rows) - report-format.md: fix stale "Phase 5, step 7" reference - shared-phases.md: fix "critical chain" → "critical path" terminology (2 occurrences)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Adds three DAG-aligned features to the agent-team-plugin, all powered by a centralized
task-graph.jsonworkspace file:compute-critical-path.shhook recomputes the longest dependency chain after every task completion. Leads prioritize critical-path blockers over non-critical work. Includes cycle detection and tie-breaking.detect-resume.shhook detects incomplete workspaces at session start with smart staleness validation (epoch-second git timestamp comparison). Users choose resume or start fresh.check-integration-point.shhook detects when convergence points (diamond dependencies) become fully unblocked and nudges the lead to verify interface compatibility.New files (6)
scripts/compute-critical-path.sh— TaskCompleted hook (critical path)scripts/detect-resume.sh— SessionStart hook (resume detection)scripts/check-integration-point.sh— TaskCompleted hook (integration checkpoints)tests/hooks/test-compute-critical-path.sh— 13 assertionstests/hooks/test-detect-resume.sh— 10 assertionstests/hooks/test-check-integration-point.sh— 11 assertionsModified files (19)
hooks/hooks.json— 3 new hook entries (9 total)docs/shared-phases.md— Phase 1b convergence points, Phase 2 critical path display, Phase 3 resume detection + task-graph.json creation, Phase 4 Critical Path Awarenessdocs/workspace-templates.md— task-graph.json schema, CP column in tasks.md templatedocs/coordination-patterns.md— Resume from Existing Workspace + Integration Checkpoint Response patterns, synced Lead Processing tabledocs/coordination-advanced.md— Deadline Escalation critical-path accelerationdocs/report-format.md— Critical path metrics, CP column in Task Ledgerskills/*/SKILL.md— step 4a reference, agent-implement convergence gateREADME.md— workspace tree, 3 hook descriptions, plugin structureCLAUDE.md— hook/script counts, verification scenarios, test countsCHANGELOG.md— v2.6.0 entry.claude-plugin/plugin.json+marketplace.json— version 2.6.0tests/lib/test-helpers.sh— task-graph mock + stdout capture helperstests/structure/test-doc-references.sh— DAG reference assertionsTest plan
claude plugin validate .passes